Skip to content

Change in case we enable the sec vtx in online#13770

Merged
ktf merged 1 commit intoAliceO2Group:devfrom
chiarazampolli:k0s-qc-part3
Dec 6, 2024
Merged

Change in case we enable the sec vtx in online#13770
ktf merged 1 commit intoAliceO2Group:devfrom
chiarazampolli:k0s-qc-part3

Conversation

@chiarazampolli
Copy link
Collaborator

@chiarazampolli chiarazampolli commented Dec 4, 2024 •
edited
Loading

In online, the strangeness tracking would be off via export ARGS_EXTRA_PROCESS_o2_secondary_vertexing_workflow='--disable-cascade-finder --disable-3body-finder --disable-strangeness-tracker but then the ASYNC part of the FST would fail.

See AliceO2Group/O2DPG#1834

@chiarazampolli chiarazampolli requested a review from a team as a code owner December 4, 2024 15:45
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2024

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1

@chiarazampolli chiarazampolli mentioned this pull request Dec 4, 2024
Merged
@alibuild
Copy link
Collaborator

alibuild commented Dec 5, 2024

Error while checking build/O2/fullCI_slc9 for b8b23bf at 2024-12-05 13:02:

## sw/BUILD/O2-latest/log
ninja: build stopped: subcommand failed.

Full log here.

chiarazampolli
chiarazampolli commented Dec 5, 2024
View reviewed changes
@chiarazampolli
Change in case we enable the sec vtx in online
4aedc0f 
In online, the strangeness tracking would be off via
export ARGS_EXTRA_PROCESS_o2_secondary_vertexing_workflow='--disable-cascade-finder --disable-3body-finder --disable-strangeness-tracker
but then the ASYNC part of the FST would fail.
@chiarazampolli
Copy link
Collaborator Author

chiarazampolli commented Dec 6, 2024

Hello @shahor02 ,

Is there a merge conflict? This is what is reported by fullCI_slc9, but I don't see any log.

Chiara

@shahor02
Copy link
Collaborator

shahor02 commented Dec 6, 2024

I guess there is a problem with the CI builder, pinging to @ktf

@ktf ktf merged commit aa4898e into AliceO2Group:dev Dec 6, 2024
7 checks passed
@ktf
Copy link
Member

ktf commented Dec 6, 2024

@singiamtel could you have a look when you are back? It looks like the new tester machine is confused.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shahor02 shahor02 shahor02 left review comments

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@chiarazampolli @alibuild @shahor02 @ktf